Skip to content

DevOps: Adding old artifact cleanup for self-hosted runners#4909

Merged
algorandskiy merged 2 commits intoalgorand:masterfrom
algojack:cleaning-up-workspace
Dec 15, 2022
Merged

DevOps: Adding old artifact cleanup for self-hosted runners#4909
algorandskiy merged 2 commits intoalgorand:masterfrom
algojack:cleaning-up-workspace

Conversation

@algojack
Copy link
Copy Markdown
Contributor

@algojack algojack commented Dec 14, 2022

We noticed that our self hosted runners are accumulating artifacts. Fixing that.

Tested on both normal and nightly tests
https://app.circleci.com/pipelines/github/algorand/go-algorand/11250/workflows/880c535b-74e1-42ab-9e1e-1db7906546df (only flaky test is failing)

@algojack algojack self-assigned this Dec 14, 2022
@algojack algojack changed the title Adding old artifact cleanup for self-hosted runners DevOps: Adding old artifact cleanup for self-hosted runners Dec 14, 2022
@algojack algojack marked this pull request as ready for review December 15, 2022 16:02
@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 15, 2022

Codecov Report

Merging #4909 (0234965) into master (9c72c4e) will decrease coverage by 0.27%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4909      +/-   ##
==========================================
- Coverage   53.65%   53.38%   -0.28%     
==========================================
  Files         430      430              
  Lines       53529    53529              
==========================================
- Hits        28723    28577     -146     
- Misses      22576    22689     +113     
- Partials     2230     2263      +33     
Impacted Files Coverage Δ
data/transactions/payset.go 50.00% <0.00%> (-50.00%) ⬇️
network/topics.go 77.08% <0.00%> (-20.84%) ⬇️
ledger/roundlru.go 69.81% <0.00%> (-20.76%) ⬇️
network/peersheap.go 88.23% <0.00%> (-5.89%) ⬇️
data/transactions/transaction.go 38.06% <0.00%> (-4.84%) ⬇️
ledger/voters.go 68.65% <0.00%> (-4.48%) ⬇️
crypto/merklearray/merkle.go 84.51% <0.00%> (-3.88%) ⬇️
crypto/stateproof/verifier.go 78.94% <0.00%> (-3.51%) ⬇️
ledger/evalindexer.go 46.91% <0.00%> (-2.47%) ⬇️
agreement/cryptoVerifier.go 65.49% <0.00%> (-2.12%) ⬇️
... and 13 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@algorandskiy algorandskiy merged commit 8cb6779 into algorand:master Dec 15, 2022
@algojack algojack deleted the cleaning-up-workspace branch December 15, 2022 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants